Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add localization for UI strings in Simplified Chinese #11723

Merged
merged 2 commits into from
Apr 12, 2022

Conversation

stepankuzmin
Copy link
Contributor

Hey 👋

This PR adds translations for UI strings we use in GL JS into Simplified Chinese. We translated the UI strings automatically using the Google Translate API, and we need to verify that the translation is correct.

To help us verify the translations, you can open the Files changed tab on this page and compare the translations with the original file src/ui/default_locale.js.

If the translation looks good, you can approve it using the Review changes button in the Files changed tab and selecting the Approve radio button when submitting the review.

To improve the machine translation, you can click the plus sign next to the line you want to enhance and select the suggestion button on the toolbar.

After making the suggestions, you can mark the review as finished using the Review changes button and selecting the Request changes radio button.

Thanks for helping 🙌

@stepankuzmin stepankuzmin added the skip changelog Used for PRs that do not need a changelog entry label Apr 11, 2022
src/ui/locales/zh-Hans.js Outdated Show resolved Hide resolved
src/ui/locales/zh-Hans.js Outdated Show resolved Hide resolved
src/ui/locales/zh-Hans.js Outdated Show resolved Hide resolved
src/ui/locales/zh-Hans.js Outdated Show resolved Hide resolved
@stepankuzmin stepankuzmin changed the base branch from main to l10n-ui April 12, 2022 10:15
Co-authored-by: Ada <[email protected]>
Co-authored-by: Peng Liu <[email protected]>
@stepankuzmin stepankuzmin requested a review from AdaXL April 12, 2022 12:54
@stepankuzmin stepankuzmin merged commit e87bd90 into l10n-ui Apr 12, 2022
@stepankuzmin stepankuzmin deleted the l10n-zh-Hans branch April 12, 2022 15:39
src/ui/locales/zh-Hans.js Show resolved Hide resolved
"GeolocateControl.LocationNotAvailable": "位置不可用",
"LogoControl.Title": "Mapbox 徽标",
"Map.Title": "地图",
"NavigationControl.ResetBearing": "将方位重置为北",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does Bearing mean?

The Chinese translation means "Reset the location to North"

Copy link

@AdaXL AdaXL Apr 15, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Jing-flyloveyin Bearing is the direction you're facing.

For you reference, this is the original English strings to compare with https://github.com/mapbox/mapbox-gl-js/blob/main/src/ui/default_locale.js

Also, #11733 for more context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip changelog Used for PRs that do not need a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants